-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Updates GitHub Actions setup-python #184
base: master
Are you sure you want to change the base?
Conversation
Awesome work on this repo! I thought it'd be cool to help try to get the CI/CD pipelines working again. The last failed one is too old to show the log anymore, so I opened this PR to get a new run of the CI/CD pipeline that then would show an error to further troubleshoot. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM thanks!
Cool! Looks like "this workflow requires approval from a maintainer" - can you kick off the CI/CD? |
Hmm, I was expecting the test workflow to run. Maybe I can get it to run in PRs. |
Thanks, that ran the test workflow and showed the syntax for updating |
The last workflow error is a known issue with old versions of |
We're still getting workflow errors, but at least a new error now. This one might get resolved by updating |
Wow, the tests ran! |
Seems like there's an issue within |
OK just send a message here if you want me to come back and approve/rerun. It was a fun project while i was working on it and i'm happy that you have interest in it. I think most of the coding work here was to develop the data infrastructure and WoW UI interfacing; but there's a lot of potential to plug in different policies |
Proposed Changes
setup-python
action for CI/CD pipeline